Skip to content

Fix/hash collision error messaging#490

Merged
LukeStampfli merged 2 commits intodevelopfrom
fix/hash-collision-error-messaging
Feb 24, 2021
Merged

Fix/hash collision error messaging#490
LukeStampfli merged 2 commits intodevelopfrom
fix/hash-collision-error-messaging

Conversation

@LukeStampfli
Copy link
Copy Markdown
Contributor

This is just a really small quality of live improvement. The error message now tells you what you are doing wrong instead of just saying something is wrong.

@unity-cla-assistant
Copy link
Copy Markdown

unity-cla-assistant commented Feb 22, 2021

CLA assistant check
All committers have signed the CLA.

@LukeStampfli LukeStampfli changed the base branch from master to develop February 22, 2021 16:28
@LukeStampfli LukeStampfli force-pushed the fix/hash-collision-error-messaging branch from 39ab294 to 4d3aed4 Compare February 22, 2021 16:32
Copy link
Copy Markdown
Contributor

@JesseOlmer JesseOlmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@LukeStampfli LukeStampfli merged commit a5f0e66 into develop Feb 24, 2021
jeffreyrainy pushed a commit that referenced this pull request Feb 24, 2021
…erator is causing the collision. (#490)

Co-authored-by: M. Fatih MAR <mfatihmar@gmail.com>
@0xFA11 0xFA11 deleted the fix/hash-collision-error-messaging branch March 5, 2021 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants